-
Notifications
You must be signed in to change notification settings - Fork 1.6k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Dataflow: Delete the old configuration-class based api. #18191
Dataflow: Delete the old configuration-class based api. #18191
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ruby / C# 👍
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Go 👍🏻
92c1c44
to
fce737f
Compare
--- | ||
category: breaking | ||
--- | ||
* Deleted the old deprecated data flow api that was based on extending a configuration class. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Something like this will help any remaining users migrate.
* Deleted the old deprecated data flow api that was based on extending a configuration class. | |
* Deleted the old deprecated data flow API that was based on extending a configuration class. See https://github.blog/changelog/2023-08-14-new-dataflow-api-for-writing-custom-codeql-queries for instructions on migrating your queries to use the new API. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
done
91f4c6a
to
aee18c7
Compare
aee18c7
to
8c99ad4
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
C++/Swift 👍
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Python 👍 we are just deleting several deprecated classes :-)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Go (again, as there were more changes) 👍🏻
Alrighty, let's merge this! |
🎉 🎉 This deletes the old and long-deprecated data flow api that was based on extending a configuration class.
See also the blog post https://github.blog/changelog/2023-08-14-new-dataflow-api-for-writing-custom-codeql-queries/ where we announced that this would happen now.